Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating expose docs to domains docs #627

Merged
merged 3 commits into from
Aug 28, 2024
Merged

Conversation

maayarosama
Copy link
Contributor

Description

Updating expose docs to domains docs

Related Issues

@Mik-TF
Copy link
Contributor

Mik-TF commented Aug 20, 2024

@maayarosama looks good in general. Some things are missing. You can check this PR #628

If you agree with my changes I'll merge the PR into your current branch (development_domains), then we can merge this PR.

Copy link
Contributor

@Mik-TF Mik-TF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good apart from some details. See this PR with some changes: #628

@Mik-TF
Copy link
Contributor

Mik-TF commented Aug 20, 2024

LGTM.

I approved the PR. As discussed with @xmonader, we will be able to merge this PR to manual.dev.grid.tf (info_grid dev branch) when the Domains app is added on the dashboard devnet (in this PR: threefoldtech/tfgrid-sdk-ts#3178)

@MohamedElmdary MohamedElmdary merged commit cb66d55 into development Aug 28, 2024
@MohamedElmdary MohamedElmdary deleted the development_domains branch August 28, 2024 13:37
@Mik-TF
Copy link
Contributor

Mik-TF commented Aug 28, 2024

I just want to make sure with the dev team that the dashboard will soon be updated on devnet to take into account Domains app. Otherwise, we would need to wait before releasing the guides on manual.dev.grid.tf (dashboard.dev.grid.tf must correspond with manual.dev.grid.tf)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants